-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
erp outputs endpoint #589
erp outputs endpoint #589
Conversation
@hdunham I tried to spin the API up locally and test the endpoints, but I'm getting a "connection refused" error. The same GET request to the staging and production servers got a good response for the existing /help endpoint. You've tested this locally? |
That's weird, yeah I did test it locally. Let me look again. |
Oh I guess checking if this ended up working for you when you looked at it again? |
…into erp-outputs-help
Yeah I've tested it many times and it's always worked |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hdunham approving, but looks like you may need to merge develop one more time to resolve merge conflicts
Please check if the PR fulfills these requirements
What kind of change does this PR introduce?
feature
What is the current behavior?
no erp/inputs (same as erp/help) or erp/outputs urls in the API
What is the new behavior (if this is a feature change)?
erp/inputs (same as erp/help) or erp/outputs urls added
Does this PR introduce a breaking change?
no
Other information:
once merged, update wiki adding
Get descriptions of possible ERP inputs: /erp/inputs?
Get descriptions of possible ERP outputs: /erp/outputs?
here:
https://github.com/NREL/REopt_API/wiki/1.-Structure-of-the-REopt-API#api-endpoints